[AN/USER] 안드로이드 네비게이션을 적용한다. (#758) #781

re4rk · 2024-03-09T01:49:35Z

📌 관련 이슈

closed: [AN] 홈 화면 네비게이션을 적용한다. #758

✨ PR 세부 내용

네비게이션 2.7.7 버전으로 적용하였습니다.
팀원들이 navigation이 없어서 설명드립니다.
- 네비게이션은 res의 navigation을 통해서 작동하게 됩니다.
- 해당 xml이 있다면 action을 통해서 다음 프래그먼트로 이동할 수 있습니다.
- 만약에 다음 프래그먼트에 argument 태그가 있다면 safeargs을 통해서 인자를 받게 됩니다.

findNavController().navigate(
    FestivalListFragmentDirections.actionFestivalListFragmentToArtistDetailFragment(
        artistId,
    ),
)

<?xml version="1.0" encoding="utf-8"?>
<navigation xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:id="@+id/festivalList"
    app:startDestination="@id/festivalListFragment">

    <fragment
        android:id="@+id/festivalListFragment"
        android:name="com.festago.festago.presentation.ui.home.festivallist.FestivalListFragment"
        android:label="FestivalListFragment"
        tools:layout="@layout/fragment_festival_list">
        <action
            android:id="@+id/action_festivalListFragment_to_artistDetailFragment"
            app:destination="@id/artistDetailFragment"
            app:enterAnim="@android:anim/slide_in_left"
            app:popExitAnim="@android:anim/slide_out_right" />
    </fragment>
    <fragment
        android:id="@+id/artistDetailFragment"
        android:name="com.festago.festago.presentation.ui.artistdetail.ArtistDetailFragment"
        android:label="ArtistDetailFragment"
        tools:layout="@layout/fragment_artist_detail">
        <argument
            android:name="artistId"
            app:argType="long" />
    </fragment>
</navigation

바텀 네비게이션의 id들이 festivalList 처럼 변하게 되었는데 해당 id가 프래그먼트와 같아야지 동작하게 되서 변경하게 되었습니다.

       - android:id="@+id/itemFestival"
       + android:id="@+id/festivalList"

전에 톡방에서 전달 드렸던 뒤로가기 버튼을 눌렀을 때 전 화면이 리프래시 되는 현상이 있는 것을 공유 드렸습니다.

해당 현상이 일어나는 이유는 navigation에서의 fragment의 생명주기가 다음과 같이 생명주기가 다시 실행되기 때문에 저희 코드에서 load를 한번 더하기 때문입니다.

- 따라서 다음 코드들이 추가 되었습니다. - 기존 binding이 있으면 기존꺼를 반환합니다. - 추가로 저희 화면마다 데이터를 호출하는데 refresh 플래그 값이 없으면 refresh 하지 않도록 변경했습니다.

# Conflicts: # android/festago/data/src/main/java/com/festago/festago/data/di/singletonscope/RepositoryModule.kt # android/festago/presentation/src/main/java/com/festago/festago/presentation/ui/home/festivallist/FestivalListFragment.kt # android/festago/presentation/src/main/res/drawable/ic_bookmark_normal.xml

# Conflicts: # android/festago/presentation/src/main/java/com/festago/festago/presentation/ui/home/festivallist/FestivalListFragment.kt # android/festago/presentation/src/main/java/com/festago/festago/presentation/ui/schooldetail/SchoolDetailFragment.kt # android/festago/presentation/src/main/res/layout/fragment_school_detail.xml

# Conflicts: # android/festago/presentation/src/main/java/com/festago/festago/presentation/ui/home/HomeActivity.kt # android/festago/presentation/src/main/java/com/festago/festago/presentation/ui/home/festivallist/FestivalListFragment.kt

SeongHoonC

상태바가 흰색으로 변경되게 만들었었는데 안되네요. backStack 이 변경되지 않아서 리스터가 동작하지 않습니다!

임시로 myPage 에 네비게이션 그래프를 추가해봤는데 스택이 따로 저장되네요! 이러면 fragment manager 4개로 관리할 필요는 없겠군요!

네비게이션이 생각보다 복잡하군요. 코멘트 확인해주세요!

SeongHoonC · 2024-03-09T09:04:07Z

...ation/src/main/java/com/festago/festago/presentation/ui/artistdetail/ArtistDetailFragment.kt

+        if (_binding == null) {
+            _binding = FragmentArtistDetailBinding.inflate(inflater)
+        }


다른 화면으로 navigate 할 때 항상 onDestroyView 가 불립니다.

네비게이션 action 은 무조건 replace 인가요?

null로 만들어주는게 onDestory인줄 알았는데 착각했네요
찾아보니 네비게이션 action은 무조건 replace로 작동하고
add처럼 동작하려면 dialog태그를 사용해야합니다.

SeongHoonC · 2024-03-09T09:08:05Z

.../src/main/java/com/festago/festago/presentation/ui/festivaldetail/FestivalDetailViewModel.kt

+    fun loadFestivalDetail(festivalId: Long, refresh: Boolean = false) {
+        if (!refresh && _uiState.value is FestivalDetailUiState.Success) return
+


예전에 기억이 잘 안나서 그런데
새로고침 요청이 아니고 Success 라면 서버에 재요청하지 않으려고 막은거였죠?

넵 initView() 호출될때마다 불필요하게 load 다시하는 거 막으려고 했던거로 기억합니당

SeongHoonC · 2024-03-09T09:10:21Z

.../festago/presentation/src/main/java/com/festago/festago/presentation/ui/home/HomeActivity.kt

+        navController =
+            (supportFragmentManager.findFragmentById(R.id.fcvHomeContainer) as NavHostFragment).navController
+        val bottomNavigationView = findViewById<BottomNavigationView>(R.id.nvHome)
+        bottomNavigationView.setupWithNavController(navController)


네비게이션 버튼을 클릭해서 이동할때는 괜찮은데 뒤로가기 누르면 홈에 쌓였던 것들이 초기화되네요..흠 왜 그런걸까요?

NavigationUI 내부적으로 첫번째 destination으로 popup 하게 되어있어서 그런 것 같아요.
bottom navigation menu item에 menuCategory를 secondary로 두면 탭 이동도 백스택 관리할 수 있어요! 아래 링크 참고
https://hide1202.github.io/android/navigation-backstack/

오오 감사합니다!

베르

사실 네비게이션 버튼 이동 시에 뒤로가기 누르면 홈에 쌓인 것들이 초기화 되는 것처럼 보이지만 그 상태에서 다시 홈을 누르면 홈의 스택이 다시 올라옵니다!
해시

네비게이션 전략은 다양하게 선택될 수 있을 것같은데 해시가 링크해준게 좋다고 생각해서 반영했습니다!

EmilyCh0

고생하셨습니다 👍 코멘트 확인해 주세요!

EmilyCh0 · 2024-03-10T06:18:05Z

...n/src/main/java/com/festago/festago/presentation/ui/festivaldetail/FestivalDetailFragment.kt

+        if (_binding == null) {
+            _binding = FragmentFestivalDetailBinding.inflate(inflater)
+        }


null이 아닌 게 어떤 경우인가요??

베르가 말해준 생명주기를 보고 롤백했습니다.

EmilyCh0 · 2024-03-10T06:29:05Z

.../src/main/java/com/festago/festago/presentation/ui/festivaldetail/FestivalDetailViewModel.kt

+    fun loadFestivalDetail(festivalId: Long, refresh: Boolean = false) {
+        if (!refresh && _uiState.value is FestivalDetailUiState.Success) return
+


넵 initView() 호출될때마다 불필요하게 load 다시하는 거 막으려고 했던거로 기억합니당

EmilyCh0 · 2024-03-10T06:47:35Z

.../festago/presentation/src/main/java/com/festago/festago/presentation/ui/home/HomeActivity.kt

+        navController =
+            (supportFragmentManager.findFragmentById(R.id.fcvHomeContainer) as NavHostFragment).navController
+        val bottomNavigationView = findViewById<BottomNavigationView>(R.id.nvHome)
+        bottomNavigationView.setupWithNavController(navController)


NavigationUI 내부적으로 첫번째 destination으로 popup 하게 되어있어서 그런 것 같아요.
bottom navigation menu item에 menuCategory를 secondary로 두면 탭 이동도 백스택 관리할 수 있어요! 아래 링크 참고
https://hide1202.github.io/android/navigation-backstack/

EmilyCh0 · 2024-03-10T06:50:18Z

...esentation/src/main/java/com/festago/festago/presentation/ui/schooldetail/SocialMediaView.kt

이 코드 제가 제거했던 것 같은데 다시 살아났네요..?

해시가 작업한 커밋이 PR에 포함되어있네요..?

병렬 작업을 위해 해시 PR 브랜치로 타겟으로 했어요
분기 후에 코드 제거를해서 그런지 살아있는거 같네요

EmilyCh0 · 2024-03-10T06:52:13Z

android/festago/presentation/src/main/res/menu/menu_bottom_navigation.xml

@@ -1,27 +1,27 @@
 <?xml version="1.0" encoding="utf-8"?>
 <menu xmlns:android="http://schemas.android.com/apk/res/android">
    <item
-        android:id="@+id/itemFestival"
+        android:id="@+id/festivalList"


#781 (comment)

SeongHoonC

프래그먼트 정책 변경 및 버그 수정 확인했습니다!
축제 목록 to 축제 상세
animation 만 추가하고 머지합니다!

SeongHoonC · 2024-03-13T02:09:25Z

.../festago/presentation/src/main/java/com/festago/festago/presentation/ui/home/HomeActivity.kt

-                is SchoolDetailFragment,
-                -> false
+    private fun initDestinationChangedListener() {
+        navController.addOnDestinationChangedListener { _, destination, _ ->


addOnDestinationChangedListener 도 있군요! 또 배워갑니다!

EmilyCh0 and others added 30 commits February 9, 2024 23:00

feat: 학교 상세 도메인 모델 추가

64e8759

feat: SchoolRepository 정의

8469319

feat: 테스트용 FakeSchool 추가

5ef793a

feat: SchoolDefaultRepository 임시 구현 (Fake 데이터 리턴)

2aa3b2b

feat: 학교 상세 dto 추가

61d0ffe

feat: SchoolDetailUiState 정의

b48c2bf

feat: SchoolDetailViewModel 구현

6eb05c6

feat: SchoolDetailFragment 구현

d6cd393

feat: UiState 분리

bc6da8e

feat: FakeSchool 소셜미디어 데이터 추가

d1f1f2b

feat: SchoolDetailFragment 뷰 수정

f74935f

move: 패키지 위치 변경

5e724f6

feat: 상단 뒤로 가기 버튼

3832327

feat: schoolId 추가

cc91707

fix: dday view 수정

a025518

feat: 북마크

b555447

fix: resolve conflict

2a85266

refactor: ktlint check

702afb4

refactor: ktlint check

48848ad

refactor: Fake 리포지토리명 변경

7a30d1c

refactor: id 카멜케이스로 수정

f1c5aec

refactor: FakeSchoolRepository 주석 삭제

52824ad

chore(presentation): 네비게이션 의존성 추가

99f08bc

feat(home): 네비게이션 그래프 추가

99f71f4

chore(presentation): safeargs 추가

15d7e1e

feat(presentation): 컨트롤러 사용하도록 추가

a20b566

feat(presentation): 탭마다 스택이 쌓이도록 변경

3c508c6

feat(presentation): 네비게이션 적용

924436c

re4rk added 4 commits March 8, 2024 20:21

refactor(Artist): 네비게이션 시 리프래시 되지 않도록 수정

a1618b0

refactor(FestivalDetail): 네비게이션 시 리프래시 되지 않도록 수정

aef1700

refactor(FestivalDetail): 불필요한 코드 제거

722c78b

re4rk linked an issue Mar 9, 2024 that may be closed by this pull request

[AN] 홈 화면 네비게이션을 적용한다. #758

Closed

refactor(FestivalDetail): 불필요한 코드 제거

1e5d425

SeongHoonC requested changes Mar 9, 2024

View reviewed changes

SeongHoonC assigned re4rk Mar 9, 2024

SeongHoonC added AN 안드로이드에 관련된 작업 USER labels Mar 9, 2024

github-actions bot requested review from EmilyCh0 and SeongHoonC March 9, 2024 17:25

EmilyCh0 approved these changes Mar 10, 2024

View reviewed changes

re4rk and others added 5 commits March 13, 2024 09:40

refactor(SchoolDetail): 사용하지 않는 모듈 제거

9794fe7

refactor(presentation): 불필요한 바인드 분기 제거

9f720ad

refactor(home): 화면마다 변경되게 변경

89b9b80

refactor(home): 화면 이동이 모두 백스택에 쌓이도록 변경

18973de

feat: 축제 목록에서 축제 상세 이동 시 애니메이션 추가

ac3c717

SeongHoonC approved these changes Mar 13, 2024

View reviewed changes

SeongHoonC merged commit 22ca63b into dev Mar 13, 2024
1 check passed

SeongHoonC deleted the feat/#758 branch March 13, 2024 02:36

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[AN/USER] 안드로이드 네비게이션을 적용한다. (#758) #781

[AN/USER] 안드로이드 네비게이션을 적용한다. (#758) #781

re4rk commented Mar 9, 2024

SeongHoonC left a comment •

edited

Loading

SeongHoonC Mar 9, 2024

re4rk Mar 13, 2024

SeongHoonC Mar 9, 2024

EmilyCh0 Mar 10, 2024

SeongHoonC Mar 9, 2024

EmilyCh0 Mar 10, 2024

SeongHoonC Mar 10, 2024

re4rk Mar 13, 2024

EmilyCh0 left a comment

EmilyCh0 Mar 10, 2024

re4rk Mar 13, 2024

EmilyCh0 Mar 10, 2024

EmilyCh0 Mar 10, 2024

EmilyCh0 Mar 10, 2024

SeongHoonC Mar 10, 2024 •

edited

Loading

re4rk Mar 13, 2024

EmilyCh0 Mar 10, 2024 •

edited

Loading

SeongHoonC left a comment

SeongHoonC Mar 13, 2024

		fun loadFestivalDetail(festivalId: Long, refresh: Boolean = false) {
		if (!refresh && _uiState.value is FestivalDetailUiState.Success) return

[AN/USER] 안드로이드 네비게이션을 적용한다. (#758) #781

[AN/USER] 안드로이드 네비게이션을 적용한다. (#758) #781

Conversation

re4rk commented Mar 9, 2024

📌 관련 이슈

✨ PR 세부 내용

SeongHoonC left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

EmilyCh0 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

SeongHoonC Mar 10, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

EmilyCh0 Mar 10, 2024 • edited Loading

Choose a reason for hiding this comment

SeongHoonC left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

SeongHoonC left a comment •

edited

Loading

SeongHoonC Mar 10, 2024 •

edited

Loading

EmilyCh0 Mar 10, 2024 •

edited

Loading